Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ja] Temporarily patch page to avoid link-check failure #5609

Merged
merged 2 commits into from
Nov 16, 2024

Conversation

katzchang
Copy link
Contributor

Fix for #5423

@katzchang katzchang requested a review from a team as a code owner November 14, 2024 11:45
@opentelemetrybot opentelemetrybot requested review from a team November 14, 2024 11:45
@katzchang
Copy link
Contributor Author

$ npm run fix:htmltest-config

> fix:htmltest-config
> scripts/htmltest-config.sh

Warning: Unrecognized htmltest IgnoreDirs config from front matter in file 'content/pt/docs/_index.md': IgnoreDirs: {}

When I try to run fix:htmltest-config, I get this error. After deleting the IgnoreDirs: {} statement in content/pt/docs/_index.md, the problem goes away.

@chalin Do you think okay to delete it?

@chalin chalin self-assigned this Nov 15, 2024
@chalin
Copy link
Contributor

chalin commented Nov 15, 2024

That should have been caught earlier. I need to fix the CI scripts first. Let me do that before approving this PR.

@chalin
Copy link
Contributor

chalin commented Nov 15, 2024

Waiting for the following to get merged, then we'll rebase this PR:

@chalin chalin force-pushed the fix-ja-library-check branch from 200bdd1 to 4ff6d53 Compare November 15, 2024 16:02
@chalin
Copy link
Contributor

chalin commented Nov 15, 2024

/fix:htmltest-config

@opentelemetrybot
Copy link
Collaborator

You triggered fix:htmltest-config action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/11859592839

@opentelemetrybot
Copy link
Collaborator

fix:htmltest-config was successful.

IMPORTANT: (RE-)RUN /fix:all to ensure that there are no remaining check issues.

@chalin
Copy link
Contributor

chalin commented Nov 15, 2024

/fix:all

@opentelemetrybot
Copy link
Collaborator

@opentelemetrybot
Copy link
Collaborator

fix:all was successful.

IMPORTANT: (RE-)RUN /fix:all to ensure that there are no remaining check issues.

@chalin chalin force-pushed the fix-ja-library-check branch from 7680db3 to 21febe5 Compare November 15, 2024 16:17
@chalin chalin enabled auto-merge November 16, 2024 15:03
@chalin chalin force-pushed the fix-ja-library-check branch from 21febe5 to 14c5d45 Compare November 16, 2024 15:04
@chalin chalin force-pushed the fix-ja-library-check branch from 14c5d45 to cecd2c1 Compare November 16, 2024 15:19
@chalin chalin added this pull request to the merge queue Nov 16, 2024
Merged via the queue into open-telemetry:main with commit a2dbe3c Nov 16, 2024
17 checks passed
drewby pushed a commit to drewby/opentelemetry.io that referenced this pull request Nov 21, 2024
drewby pushed a commit to drewby/opentelemetry.io that referenced this pull request Dec 5, 2024
ymotongpoo pushed a commit to ymotongpoo/opentelemetry.io that referenced this pull request Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants